Skip to content

Report Tangent editor reconnect failures once per outage - #3017

Open
camielvs wants to merge 1 commit into
masterfrom
fix-editor-reconnect-notifications
Open

camielvs wants to merge 1 commit into
masterfrom
fix-editor-reconnect-notifications

Conversation

@camielvs

@camielvs camielvs commented Nov 6, 2026

Copy link
Copy Markdown
Collaborator

Summary

Report browser editor-control reconnect errors once per connection outage instead of notifying on every Socket.IO connect_error (websocket/xhr polling retries).

Reset the notification latch on successful reconnect. Preserve initial connection rejection, automatic retry behavior, stale-client suppression and existing diagnostic logging. Each host reports independently; no global state is introduced.

Validation

32 focused tests pass across remoteEnvHost and connectRemoteEnvWithRefresh. TypeScript and targeted ESLint pass. Added tests for repeated mixed transport errors, recovery/new outage, stale disconnect callbacks and initial connect failure.

Scope

Confirmed the repeated-notification code path while investigating production project 01a1b305029cbb40e304. This prevents retry spam; it does not claim to repair the underlying transport/proxy failure, which has not been reproduced. A polling handshake to the project's proxy succeeded during read-only investigation.

@camielvs
camielvs requested a review from a team as a code owner November 6, 2026 23:19
@github-actions

github-actions Bot commented Nov 6, 2026

Copy link
Copy Markdown

🎩 Preview

A preview build has been created at: fix-editor-reconnect-notifications/55b7152

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant